Skip to content

fix(legacy): skip preload helper in legacy chunks#21607

Merged
sapphi-red merged 1 commit into
vitejs:mainfrom
sapphi-red:fix/legacy-skip-preload-helper-in-legacy-chunks
Feb 10, 2026
Merged

fix(legacy): skip preload helper in legacy chunks#21607
sapphi-red merged 1 commit into
vitejs:mainfrom
sapphi-red:fix/legacy-skip-preload-helper-in-legacy-chunks

Conversation

@sapphi-red
Copy link
Copy Markdown
Member

Since the format conversion is done in the plugin, this condition is true for the legacy chunks as well.

const isModern = String(format === 'es')

This PR fixes that problem by replacing the modern flag in the legacy plugin.

@sapphi-red sapphi-red added plugin: legacy p3-minor-bug An edge case that only affects very specific usage (priority) labels Feb 10, 2026
@sapphi-red sapphi-red merged commit 0f2b7ae into vitejs:main Feb 10, 2026
21 checks passed
@sapphi-red sapphi-red deleted the fix/legacy-skip-preload-helper-in-legacy-chunks branch February 10, 2026 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p3-minor-bug An edge case that only affects very specific usage (priority) plugin: legacy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant